Skip to content

V22 gas changes #717

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

RdeWilde
Copy link
Contributor

@RdeWilde RdeWilde commented May 6, 2025

Fixes #716


name: V22 gas changes
about: Recent changes in v22 protocol
labels:


IMPORTANT NOTICE:
I read and understood the guidelines for contributions to the TRD. The contribution may qualify for being compensated by the TRD grant if approved by the maintainers.

This PR resolves the issue . The following steps were performed:

  • Analysis: If the described issue is a bug report, analyze the reasons resulting in this bug.

  • Solution: Describe the proposed solution for the bug or feature.

  • Implementation: Rough description/explanation of the implementation choices.

  • Performed tests: Describe the performed tests.

  • Documentation: Make sure to document the added changes in a proper way (Readme, help section, documentation, comments in code if needed)

  • Check list:

  • I extended the Github Actions CI test units with the corresponding tests for this new feature (if needed).
  • I extended the Sphinx documentation (if needed).
  • I extended the help section (if needed).
  • I changed the README file (if needed).
  • I created a new issue if there is further work left to be done (if needed).

Work effort: Give your estimate of the work effort in hours. This might be adjusted or discussed by the other contributors in order to keep a fair rewarding process for the efforts.

RdeWilde added 2 commits May 2, 2025 14:53
As mentioned by borsss on Discord https://discord.com/channels/699325006928281720/1324419368079200297/1367843233148829877 minimum gas limit has increased for the v22 protocol change, and therefore should be increase here also.



Signed-off-by: R. de Wilde <[email protected]>
As suggested on discord because of changed gas As suggested on Discord: https://discord.com/channels/699325006928281720/1324419368079200297/1367873516279562322

Signed-off-by: R. de Wilde <[email protected]>
@nicolasochem nicolasochem merged commit 9e0ab88 into tezos-reward-distributor-organization:master May 10, 2025
2 of 8 checks passed
@nicolasochem
Copy link
Contributor

Thanks! I merged and made you maintainer. Also consider doing something like this for Rio: 42bb7d6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error while validating operation - Status: failed, Message: proto.022-PsRiotum.gas_exhausted.operation
2 participants